-
Notifications
You must be signed in to change notification settings - Fork 44
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PIPRES-261: Restrict BO subscription list for all shops #823
Merged
mandan2
merged 2 commits into
PIPRES-261-separate-carrier-for-recurring-orders-improvements
from
PIPRES-261-restrict-bo-subscription-list-for-all-shops
Oct 3, 2023
Merged
PIPRES-261: Restrict BO subscription list for all shops #823
mandan2
merged 2 commits into
PIPRES-261-separate-carrier-for-recurring-orders-improvements
from
PIPRES-261-restrict-bo-subscription-list-for-all-shops
Oct 3, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
mandan2
commented
Oct 3, 2023
mandan2
merged commit Oct 3, 2023
74197b5
into
PIPRES-261-separate-carrier-for-recurring-orders-improvements
5 checks passed
mandan2
deleted the
PIPRES-261-restrict-bo-subscription-list-for-all-shops
branch
October 3, 2023 09:12
tomjas1997
reviewed
Oct 3, 2023
|
||
if ($shop->getContext() !== \Shop::CONTEXT_SHOP) { | ||
if (!$this->get('session')->getFlashBag()->has('error')) { | ||
$this->addFlash('error', $this->module->l('Select the shop that you want to configure')); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pretty sure trans should be used here?
mandan2
added a commit
that referenced
this pull request
Oct 3, 2023
* PIPRES-261: shipping option module setting (#814) * PIPRES-261: Shipping option module setting * csfixer * install and uninstall subscription configuration * improved faq page * PIPRES-333: Missing registered interface (#816) * PIPRES-261: Get carrier price to create subscription (#817) * PIPRES-261: Get carrier price to create subscription * updated subscription faq * renamed some services and added additional conditions for carrier retrieve * addedd back to exception codes * PIPRES-261: Set carrier for recurring order (#818) * PIPRES-261: Get carrier price to create subscription * renamed some services and added additional conditions for carrier retrieve * PIPRES-261: Set carrier for recurring order * PIPRES-261: Subscription order detail view refactoring (#819) * PIPRES-261: Get carrier price to create subscription * renamed some services and added additional conditions for carrier retrieve * addedd back to exception codes * PIPRES-261: Subscription order detail view refactoring * phpstan * PIPRES-261: Minor improvements (#820) * PIPRES-261: Get carrier price to create subscription * renamed some services and added additional conditions for carrier retrieve * PIPRES-261: Minor improvements * PIPRES-261: Create fresh specific price (#822) * PIPRES-261: Restrict BO subscription list for all shops (#823) * PIPRES-261: Restrict BO subscription list for all shops * fixed redundant error mesage
mandan2
added a commit
that referenced
this pull request
Oct 16, 2023
* PIPRES-113: Billie payment method (#784) * PIPRES-113: Billie payment method validation (#783) * PIPRES-113: Billie payment method validation * added missing services to base service provider * wrong method call fix * missing repository * PIPRES-113: Billie payment method setup (#788) * PIPRES-113: Billie payment method additional data send (#789) * PIPRES-316: hookActionFrontControllerAfterInit replacement on deprecated versions (#797) * Version bump and upgrade file fix (#800) * release-6.0.1. * updating the e2e on PR yml targeting `develop` branch for E2E testing * adding Ngrok launch fix preventing the failing CI, while Ngrok is busy * small yml fix * fixing the module upgrade CI testing * small fix * upgrading text fix * test fixes * small fix * removing the PS8 CI upgrading test because 5.2.0 was incompatible with PS8x * cypress upgrading * useful E2E test spec updates * adding improved Cypress terminal logs * debug mode enabling * Update prestashop_1784_2.sql * useful cypress updates * useful test code updates * specs updates * ps8 test spec updates * Update Makefile * test spec updates * updates for test specs * updates for test specs * Update 02_mollie.ps1784.EnablingPaymentsOrdersAPI.specs.js * ps8 test code updates * ps1784 test spec updates * in3 payment wip * Update 03_mollie.ps8.Subscriptions.WIP.js * Update .htaccess1784 * Update prestashop_1784_2.sql * ps1784 test updates * Update 03_mollie.ps1784.PaymentTests.js * ps1784 test updates * Update 03_mollie.ps1784.PaymentTests.js * Update .gitignore * ps1784 finalized updates * disabling debug mode * Revert "disabling debug mode" This reverts commit 0f70ace. * spec update * spec updates * spec updates (ps1784/ps8) * Update 04_mollie.ps8.Subscriptions.WIP.js * Update 02_mollie.ps8.EnablingPaymentsOrdersAPI.specs.js * Update Makefile * Update prestashop_8.sql * Update prestashop_8.sql * Update .htaccess8 * Update prestashop_8.sql * Update 02_mollie.ps8.EnablingPaymentsOrdersAPI.specs.js * spec updates * ps8 test spec updates * ps8 small typo fixes * Update 03_mollie.ps8.PaymentTests.js * cypress updates * Update 03_mollie.ps1784.PaymentTests.js * Update cypress.config.js * adding ps8.0.5 docker build * Update .htaccess8 * Update 03_mollie.ps8.PaymentTests.js * Update 03_mollie.ps8.PaymentTests.js * ps1784 debug mode disabling * ps8 debug mode disabling * small update * small typo fix * Update E2E_On_PR.yml * adding random branch string ending * Update upgrading_check.yml * Update cypress.config.js * selector updates * updating the testrail extension by Chris * temporary disabling the error checking * increasing timeout for selectors * cypress-testrail extension fix * PIPRES-261: Mixed cart checkout for subscriptions (#803) * PIPRES-261: New order split into separate subscription order * csfixer * stan * improved validations * removed commented code * note fixes * test fix * stan fix * reduced redundant code * PIPRES-261: Payment attribute fix (#806) * PIPRES-261: Payment attribute fix * changelog * replaced getProducts to getCartProducts and product attribute array key * PIPRES-306: Configuration updateValue multishop context fix (#808) (#811) * PIPRES-306: Configuration updateValue multishop context fix * tests fix * stan fix * PIPRES-338: Mixed cart subscription listing improvements (#812) * PIPRES-338: Mixed cart subscription listing improvements * added rounding for total price and moved part of install improvements to main install file * fix * added mandate_id append to upgrade file too * fix * PIPRES-113: Billie payment method additional check for vat number (#813) * ps1784 debug mode * updating terminal report extension * ps8 debug mode * Update Dockerfile.1784 * upgrading the build to PS1785 * ps1785 updates * ps1785 updates * Update E2E_On_PR.yml * moving things to ps1785 * Update E2E_On_PR.yml * Update E2E_On_PR.yml * downgrading cypress breaking changes in the terminal module, temporary downgrading Cypress * some e2e fixes * Update 04_mollie.ps1785.Subscriptions.js * Update 03_mollie.ps8.PaymentTests.js * Update 04_mollie.ps8.Subscriptions.js * adding testrail ids for CI * Update 03_mollie.ps8.PaymentTests.js * small typo * small e2e improvements * PIPRES-261: Separate carrier for recurring orders improvements (#821) * PIPRES-261: shipping option module setting (#814) * PIPRES-261: Shipping option module setting * csfixer * install and uninstall subscription configuration * improved faq page * PIPRES-333: Missing registered interface (#816) * PIPRES-261: Get carrier price to create subscription (#817) * PIPRES-261: Get carrier price to create subscription * updated subscription faq * renamed some services and added additional conditions for carrier retrieve * addedd back to exception codes * PIPRES-261: Set carrier for recurring order (#818) * PIPRES-261: Get carrier price to create subscription * renamed some services and added additional conditions for carrier retrieve * PIPRES-261: Set carrier for recurring order * PIPRES-261: Subscription order detail view refactoring (#819) * PIPRES-261: Get carrier price to create subscription * renamed some services and added additional conditions for carrier retrieve * addedd back to exception codes * PIPRES-261: Subscription order detail view refactoring * phpstan * PIPRES-261: Minor improvements (#820) * PIPRES-261: Get carrier price to create subscription * renamed some services and added additional conditions for carrier retrieve * PIPRES-261: Minor improvements * PIPRES-261: Create fresh specific price (#822) * PIPRES-261: Restrict BO subscription list for all shops (#823) * PIPRES-261: Restrict BO subscription list for all shops * fixed redundant error mesage * url updates * refactoring PS17 tests * ps8 tests refactored * Update 04_mollie.ps1785.EnablingPaymentsPaymentsAPI.specs.js * updating urls * small typo * small typo --------- Co-authored-by: SimonasB88 <[email protected]>
mandan2
added a commit
that referenced
this pull request
Oct 17, 2023
* PIPRES-261: shipping option module setting (#814) * PIPRES-261: Shipping option module setting * csfixer * install and uninstall subscription configuration * improved faq page * PIPRES-333: Missing registered interface (#816) * PIPRES-261: Get carrier price to create subscription (#817) * PIPRES-261: Get carrier price to create subscription * updated subscription faq * renamed some services and added additional conditions for carrier retrieve * addedd back to exception codes * PIPRES-261: Set carrier for recurring order (#818) * PIPRES-261: Get carrier price to create subscription * renamed some services and added additional conditions for carrier retrieve * PIPRES-261: Set carrier for recurring order * PIPRES-261: Subscription order detail view refactoring (#819) * PIPRES-261: Get carrier price to create subscription * renamed some services and added additional conditions for carrier retrieve * addedd back to exception codes * PIPRES-261: Subscription order detail view refactoring * phpstan * PIPRES-261: Minor improvements (#820) * PIPRES-261: Get carrier price to create subscription * renamed some services and added additional conditions for carrier retrieve * PIPRES-261: Minor improvements * PIPRES-261: Create fresh specific price (#822) * PIPRES-261: Restrict BO subscription list for all shops (#823) * PIPRES-261: Restrict BO subscription list for all shops * fixed redundant error mesage
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.